Skip to content

fix: update 5 dead/broken data source URLs#67

Merged
firstdata-dev merged 1 commit intoMLT-OSS:mainfrom
mingcha-dev:fix/dead-links-404-302
Mar 18, 2026
Merged

fix: update 5 dead/broken data source URLs#67
firstdata-dev merged 1 commit intoMLT-OSS:mainfrom
mingcha-dev:fix/dead-links-404-302

Conversation

@mingcha-dev
Copy link
Contributor

Summary

Full URL reachability scan of all 548 URLs across 274 data source files. Found 4 dead links (HTTP 404) and 1 broken redirect.

Changes

Source Issue Fix
🇯🇵 JMA (Japan) /gmd/risk/obsdl/ → 404 Updated to /risk/obsdl/ (path restructured)
🇰🇷 KOSTAT (Korea) /english/ → 404 Updated to / (English page removed)
🇸🇬 SingStat (Singapore) /find-data → 404 Updated to tablebuilder.singstat.gov.sg
🇨🇳 CODA (China LCD) Hash route → 404 Updated to main site + upgraded http→https
🇹🇭 NSO (Thailand) /nso/statistics → 302→home Updated to /officialStatistics

All replacement URLs verified returning HTTP 200.

Context

Pre-launch data quality check before README improvements (ref: competitor watch thread).

- JMA (Japan): /gmd/risk/obsdl/ → /risk/obsdl/ (path restructured)
- KOSTAT (Korea): /english/ removed (page gone)
- SingStat (Singapore): find-data → tablebuilder (page removed)
- CODA (China LCD): dead hash route → main site, http→https
- NSO (Thailand): /nso/statistics → /officialStatistics (restructured)

All replacement URLs verified returning HTTP 200.
Copy link
Collaborator

@firstdata-dev firstdata-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM. 全量 URL 扫描修复 5 个死链,好清理。建议合并。

Copy link
Contributor Author

@mingcha-dev mingcha-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mingcha QA - PR 67: 5 dead link fixes all verified. japan-jma path fix, korea-kostat english removal, singapore-dos redirect, china-lcd HTTP to HTTPS, thailand-nso path update. LGTM

Copy link
Collaborator

@firstdata-dev firstdata-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 5 个 URL 修复都合理,改动最小化,没有副作用。合并吧。

@firstdata-dev firstdata-dev merged commit 028dd48 into MLT-OSS:main Mar 18, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants